Renable compiler warnings and fix a few that show up

The switch to GNOME_COMPILE_WARNINGS() caused -Wall and other
warnings to not actually be used since GNOME_COMPILE_WARNINGS()
just sets WARN_CFLAGS. Add WARN_CFLAGS to AM_CFLAGS so that
it takes effect.

Add  -Wno-error=deprecated-declarations so that when -Werror is
enabled, we don't fail on all the deprecated cogl and clutter
symbols.

https://bugzilla.gnome.org/show_bug.cgi?id=730408

diff --git a/configure.ac b/configure.ac
index 40ed129e0e44571ccf21fcf68ecaef060f5a17d8..8b0fe94692a71f7b8a88758699ade9d796874a40 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -223,6 +223,14 @@ fi
 AM_CONDITIONAL(ENABLE_MAN, test "$enable_man" != no)
 
 GNOME_COMPILE_WARNINGS([error])
+case "$WARN_CFLAGS" in
+    *-Werror*)
+        WARN_CFLAGS="$WARN_CFLAGS -Wno-error=deprecated-declarations"
+        ;;
+esac
+
+AM_CFLAGS="$AM_CFLAGS $WARN_CFLAGS"
+AC_SUBST(AM_CFLAGS)
 
 BROWSER_PLUGIN_DIR="${BROWSER_PLUGIN_DIR:-"\${libdir}/mozilla/plugins"}"
 AC_ARG_VAR([BROWSER_PLUGIN_DIR],[Where to install the plugin to])

diff --git a/src/shell-perf-helper.c b/src/shell-perf-helper.c
index 6d955bfd51762d065a960e88931c51a0ba3cbfc4..dabaa549983f65ce424942ff61745d5957d87e5d 100644 (file)
--- a/src/shell-perf-helper.c
+++ b/src/shell-perf-helper.c
@@ -75,7 +75,7 @@ on_timeout (gpointer data)
 }
 
 static void
-establish_timeout ()
+establish_timeout (void)
 {
   if (timeout_id != 0)
     g_source_remove (timeout_id);

diff --git a/src/st/st-box-layout-child.c b/src/st/st-box-layout-child.c
index 71a30d817638cb520fb91c330d3ca11b0c987436..dab684a15546aefbed14dc92ea61c9ac3664eba1 100644 (file)
--- a/src/st/st-box-layout-child.c
+++ b/src/st/st-box-layout-child.c
@@ -94,6 +94,9 @@ st_box_layout_child_get_property (GObject    *object,
           break;
         case CLUTTER_BOX_ALIGNMENT_END:
           align = ST_ALIGN_END;
+          break;
+        default:
+          g_assert_not_reached ();
         }
       g_value_set_enum (value, align);
       break;
@@ -138,6 +141,9 @@ st_box_layout_child_set_property (GObject      *object,
           break;
         case ST_ALIGN_END:
           align = CLUTTER_BOX_ALIGNMENT_END;
+          break;
+        default:
+          g_assert_not_reached ();
         }
       g_object_set (meta, g_param_spec_get_name (pspec), align, NULL);
       break;

diff --git a/src/st/st-texture-cache.c b/src/st/st-texture-cache.c
index 6ec358165e2ab409fe3d4cb2759786cbd8bf9a20..02ab1a335ed5de3a42685784bb58ee33ef47355e 100644 (file)
--- a/src/st/st-texture-cache.c
+++ b/src/st/st-texture-cache.c
@@ -21,6 +21,8 @@
 
 #include "config.h"
 
+#define COGL_ENABLE_EXPERIMENTAL_API /* for cogl_framebuffer_allocate */
+
 #include "st-texture-cache.h"
 #include "st-private.h"
 #include <gtk/gtk.h>